-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
82 feature request suggestion create GitHub actions workflow for automating the build and deployments for feature branches #87
base: main
Are you sure you want to change the base?
82 feature request suggestion create GitHub actions workflow for automating the build and deployments for feature branches #87
Conversation
2461703
to
13a682f
Compare
Cargo.toml
Outdated
@@ -17,6 +17,7 @@ clap = { version = "4.5.20", features = ["cargo", "derive", "env"] } | |||
log = "0.4.25" | |||
simplelog = { version = "0.12.2", features = ["paris"] } | |||
tokio = "1.41.0" | |||
libsqlite3-sys = "0.30.0" # or "0.31.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lmcdonough We don't use sqlite in our backend, is there something that's calling for this from the GitHub builder?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, removed it.
entity/Cargo.toml
Outdated
@@ -8,15 +8,15 @@ name = "entity" | |||
path = "src/lib.rs" | |||
|
|||
[dependencies] | |||
axum-login = "0.16.0" | |||
axum-login = "0.17.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@lmcdonough I don't recommend updating these packages in this PR as they're not related to what you're trying to accomplish here. We need to be able to test the backend when making these version updates, so it's best to do that as a separate PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed, I will revert this to 0.16.0
… with updated actions, improved environment variable handling, and added artifact attestations for images.
…tHub Actions workflow details
…tructure - Change base image in Dockerfile to rust:1.70-slim - Use environment variable for backend port in Dockerfile - Update volume paths in docker-compose to point to the docs directory - Specify build target in docker-compose for runtime - Adjust frontend build context in docker-compose
…nstalls architecture-specific packages
…ation to within if logic
…ds entrypoint.sh as the entrypoint
… and runs the correct binary in the pulled image.
…tform support and cleanup
ae72837
to
242ac4a
Compare
…rvice readiness
… bash
…sses in env vars
This pull request introduces a new GitHub Actions workflow to build and deploy Docker containers for Rust and Next.js projects. The workflow includes steps for building, testing, and pushing Docker images to the GitHub Container Registry.
GitHub Issue: Closes #82
Changes
.github/workflows/build_and_deploy_containers.yml
to automate the build and deployment process for feature branches and pull requests to the main branch.build_test_run
for building and testing the project, andbuild_and_push_docker
for building and pushing Docker images. The latter job depends on the successful completion of the former.seaORM CLI
, build the project, and run tests.Testing Strategy
workflow_dispatch
to iterate and view any errors from the workflow run.Concerns